Skip to content

Add ubrella-clickhose-review and codebase-consistency-reviewer skills#1729

Open
filimonov wants to merge 1 commit intoantalya-26.3from
filimonov/antalya-26.3/skills
Open

Add ubrella-clickhose-review and codebase-consistency-reviewer skills#1729
filimonov wants to merge 1 commit intoantalya-26.3from
filimonov/antalya-26.3/skills

Conversation

@filimonov
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Documentation entry for user-facing changes

2 skills for claude for clickhouse developement.

Usage is simple:

/codebase-consistency-reviewer <commit range | pr number | commit ett>

this checks if your code 're-invite the bicycle' by doing the thing which was done 100 times before, and proposes rewrites.

and

/ubrella-clickhose-review <commit range | pr number | commit ett>

check the code from several different angles by several independent sub-agents, each focused on one thing, and cross-checking / summarizing their findings in one reports.

CI/CD Options

Exclude tests:

  • Fast test
  • Integration Tests
  • Stateless tests
  • Stateful tests
  • Performance tests
  • All with ASAN
  • All with TSAN
  • All with MSAN
  • All with UBSAN
  • All with Coverage
  • All with Aarch64
  • All Regression
  • Disable CI Cache

Regression jobs to run:

  • Fast suites (mostly <1h)
  • Aggregate Functions (2h)
  • Alter (1.5h)
  • Benchmark (30m)
  • ClickHouse Keeper (1h)
  • Iceberg (2h)
  • LDAP (1h)
  • Parquet (1.5h)
  • RBAC (1.5h)
  • SSL Server (1h)
  • S3 (2h)
  • S3 Export (2h)
  • Swarms (30m)
  • Tiered Storage (2h)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Workflow [PR], commit [0985961]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant